-
Notifications
You must be signed in to change notification settings - Fork 5.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: Fix pod log viewer scrollbars #14199
Conversation
Signed-off-by: Alex Collins <[email protected]>
Signed-off-by: Alex Collins <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YOLO Approve.
@crenshaw-dev can I bump review pls? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Local test confirms scrolls are scrollier.
@alexec can you check the CI lint issue? |
/cherry-pick release-2.8 |
/cherry-pick release-2.7 |
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #14199 +/- ##
==========================================
- Coverage 49.73% 49.72% -0.01%
==========================================
Files 261 261
Lines 44705 44705
==========================================
- Hits 22234 22231 -3
- Misses 20284 20286 +2
- Partials 2187 2188 +1 ☔ View full report in Codecov by Sentry. |
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <[email protected]> * fix scrolling Signed-off-by: Alex Collins <[email protected]> --------- Signed-off-by: Alex Collins <[email protected]>
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <[email protected]> * fix scrolling Signed-off-by: Alex Collins <[email protected]> --------- Signed-off-by: Alex Collins <[email protected]>
* fix: Fix pod log viewer scrollbars * fix scrolling --------- Signed-off-by: Alex Collins <[email protected]> Co-authored-by: Alex Collins <[email protected]>
* fix: Fix pod log viewer scrollbars * fix scrolling --------- Signed-off-by: Alex Collins <[email protected]> Co-authored-by: Alex Collins <[email protected]>
* fix: Fix pod log viewer scrollbars * fix scrolling --------- Signed-off-by: Alex Collins <[email protected]> Co-authored-by: Alex Collins <[email protected]> Signed-off-by: schakrad <[email protected]>
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <[email protected]> * fix scrolling Signed-off-by: Alex Collins <[email protected]> --------- Signed-off-by: Alex Collins <[email protected]>
* fix: Fix pod log viewer scrollbars Signed-off-by: Alex Collins <[email protected]> * fix scrolling Signed-off-by: Alex Collins <[email protected]> --------- Signed-off-by: Alex Collins <[email protected]>
List
. You must useGrid
.scrollToBottom
when the user tries to scroll to the bottom.